-
Notifications
You must be signed in to change notification settings - Fork 74k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dataset doc updated #27933
Dataset doc updated #27933
Conversation
``` | ||
|
||
The input signature of `map_func` is determined by the structure of each | ||
element in this dataset. For example: | ||
|
||
```python | ||
# NOTE: The following examples use `{ ... }` to represent the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do you introduce this comment here? I thought the purpose of this PR was to replace { ... }
with code that runs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are several examples following line in line 944 - 961 that I didn't change in this request and still contain non executable code. I wanted to keep this request small because I expected some feedback. I'll take another look in a separate request.
PiperOrigin-RevId: 244259677
Following up on issue #25320. This pull request clarifies the documentation by adding runnable code for several methods of the Dataset class (zip, concatenate, interleave, map, flat_map)
@mrry @jsimsa